[Security Solutions] Add tooltips #73436

cnasikas · 2020-07-28T12:21:46Z

Summary

This PR adds three tooltips. The first two are tooltips for the attaching to a case buttons. The third tooltip is for the Upload value lists button in the Detections page.

Timeline:

Detections:

Checklist

Delete any items that are not applicable to this PR.

Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
This renders correctly on smaller devices using a responsive layout. (You can test this in your browser
This was checked for cross-browser compatibility

For maintainers

This was checked for breaking API changes and was labeled appropriately

elasticmachine · 2020-07-28T12:21:48Z

Pinging @elastic/siem (Team:SIEM)

...ck/plugins/security_solution/public/timelines/components/timeline/properties/translations.ts

andrew-goldstein

Do you think we should have a different tooltip to represent the state when the actions are enabled?

For example, when the timeline does not have a title (and the actions are disabled), the tooltip prompts the user to take action, Please provide a title for your timeline in order to attach it to a case, per the screenshot below:

After taking the prescribed action to provide a title, the actions become enabled, but per the screenshot below, the tooltip still suggests Please provide a title for your timeline in order to attach it to a case, a that task has already been accomplished:

cnasikas · 2020-07-28T17:11:35Z

Do you think we should have a different tooltip to represent the state when the actions are enabled?

For example, when the timeline does not have a title (and the actions are disabled), the tooltip prompts the user to take action, Please provide a title for your timeline in order to attach it to a case, per the screenshot below:

After taking the prescribed action to provide a title, the actions become enabled, but per the screenshot below, the tooltip still suggests Please provide a title for your timeline in order to attach it to a case, a that task has already been accomplished:

You are right! I think is best to not have a tooltip when the actions become enabled.

andrew-goldstein

Thanks for this helpful tooltip @cnasikas! 🙏
Desk tested locally
LGTM 🚀

andrew-goldstein · 2020-07-28T19:52:15Z

@elasticmachine merge upstream

cnasikas · 2020-07-28T21:07:15Z

@elasticmachine merge upstream

andrew-goldstein · 2020-07-28T23:02:18Z

@elasticmachine merge upstream

andrew-goldstein · 2020-07-29T02:00:18Z

@elasticmachine merge upstream

andrew-goldstein · 2020-07-29T04:39:31Z

@elasticmachine merge upstream

kibanamachine · 2020-07-29T06:33:36Z

💚 Build Succeeded

continuous-integration/kibana-ci/pull-request
Commit: 6fd6be3

Build metrics

async chunks size

id	value	diff	baseline
securitySolution	7.3MB	+1.5KB	7.3MB

History

💔 Build #65068 failed 558b69c
💔 Build #65008 failed 80e9514
💔 Build #64941 failed 3b8d61a
💔 Build #64895 failed b03576e
💔 Build #64834 failed 62983f1

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

## Summary This PR adds three tooltips. The first two are tooltips for the `attaching to a case` buttons. The third tooltip is for the `Upload value lists` button in the `Detections` page. **Timeline:** <img width="672" alt="Screenshot 2020-07-28 at 3 08 31 PM" src="https://user-images.githubusercontent.com/7871006/88664558-e79aee00-d0e5-11ea-9644-48480c194498.png"> <img width="673" alt="Screenshot 2020-07-28 at 3 08 38 PM" src="https://user-images.githubusercontent.com/7871006/88664552-e49ffd80-d0e5-11ea-8d9e-2910f58332d4.png"> **Detections:** <img width="639" alt="Screenshot 2020-07-28 at 3 19 53 PM" src="https://user-images.githubusercontent.com/7871006/88664531-dce05900-d0e5-11ea-9dcd-7e336f215e40.png"> ### Checklist Delete any items that are not applicable to this PR. - [x] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md) - [x] This renders correctly on smaller devices using a responsive layout. (You can test this [in your browser](https://www.browserstack.com/guide/responsive-testing-on-local-server) - [x] This was checked for [cross-browser compatibility](https://www.elastic.co/support/matrix#matrix_browsers) ### For maintainers - [x] This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process)

## Summary This PR adds three tooltips. The first two are tooltips for the `attaching to a case` buttons. The third tooltip is for the `Upload value lists` button in the `Detections` page. **Timeline:** <img width="672" alt="Screenshot 2020-07-28 at 3 08 31 PM" src="https://user-images.githubusercontent.com/7871006/88664558-e79aee00-d0e5-11ea-9644-48480c194498.png"> <img width="673" alt="Screenshot 2020-07-28 at 3 08 38 PM" src="https://user-images.githubusercontent.com/7871006/88664552-e49ffd80-d0e5-11ea-8d9e-2910f58332d4.png"> **Detections:** <img width="639" alt="Screenshot 2020-07-28 at 3 19 53 PM" src="https://user-images.githubusercontent.com/7871006/88664531-dce05900-d0e5-11ea-9dcd-7e336f215e40.png"> ### Checklist Delete any items that are not applicable to this PR. - [x] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md) - [x] This renders correctly on smaller devices using a responsive layout. (You can test this [in your browser](https://www.browserstack.com/guide/responsive-testing-on-local-server) - [x] This was checked for [cross-browser compatibility](https://www.elastic.co/support/matrix#matrix_browsers) ### For maintainers - [x] This was checked for breaking API changes and was [labeled appropriately](https://github.com/elastic/kibana/blob/master/CONTRIBUTING.md#release-notes-process) Co-authored-by: Christos Nasikas <christos.nasikas@elastic.co>

Co-authored-by: Christos Nasikas <christos.nasikas@elastic.co> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

* master: (126 commits) [ML] Disabling ML if license feature is disabled (elastic#73187) [ML] Fixing old _xpack style es endpoint paths (elastic#73667) [DOCS] [Lens] 7.9 docs refresh (elastic#72301) [ML] DF Analytics results: ensure `View` link is only enabled when job has successfully completed (elastic#73539) Set timeRange to default to trigger the error message (elastic#73629) [ML] Functional tests - stabilize DFA navigation and index pattern handling (elastic#73660) [ILM] Add links to "Snapshot and Restore" from ILM "wait for snapshot policy" (elastic#72473) [kbn-storybook] Update Storybook to 5.3.19 (elastic#73320) [Metrics UI] Fix hasData call to ensure it has data not just indices (elastic#72969) [Uptime] Use `service.name` to link from Uptime -> APM where available (elastic#73618) allow others to update `URL.revokeObjectURL` property if needed (elastic#73639) regen docs (elastic#73650) [Visualize] Fix inspector download filename issue when saving in-place (elastic#72605) [Data] Query Input String manager (elastic#72093) [Security Solutions] Add tooltips (elastic#73436) Do not render descriptionless actions within an EuiCard (elastic#73611) [Security Solution][Detections] Value Lists Modal supports multiple exports (elastic#73532) [Security Solution][Resolver] Handle disabled process collection (elastic#73592) [Security_Solution][Bug] Fix user name/domain to ECS structure (elastic#73530) [Security Solution][Exceptions] - Update rule.exceptions_list to include exception list list_id (elastic#73349) ...

…ibana into actions/webhook-remove-header * 'actions/webhook-remove-header' of github.com:gmmorris/kibana: (86 commits) [maps] rename GisMap to MapContainer and convert to TS (elastic#73690) [APM] docs: remove watcher documentation (elastic#73485) [Maps] fix fit to data for Point to Point layer (elastic#73563) [Metrics UI] Fix No Data in Inventory alerts/Snapshot API (elastic#72513) [ML] Disabling ML if license feature is disabled (elastic#73187) [ML] Fixing old _xpack style es endpoint paths (elastic#73667) [DOCS] [Lens] 7.9 docs refresh (elastic#72301) [ML] DF Analytics results: ensure `View` link is only enabled when job has successfully completed (elastic#73539) Set timeRange to default to trigger the error message (elastic#73629) [ML] Functional tests - stabilize DFA navigation and index pattern handling (elastic#73660) [ILM] Add links to "Snapshot and Restore" from ILM "wait for snapshot policy" (elastic#72473) [kbn-storybook] Update Storybook to 5.3.19 (elastic#73320) [Metrics UI] Fix hasData call to ensure it has data not just indices (elastic#72969) [Uptime] Use `service.name` to link from Uptime -> APM where available (elastic#73618) allow others to update `URL.revokeObjectURL` property if needed (elastic#73639) regen docs (elastic#73650) [Visualize] Fix inspector download filename issue when saving in-place (elastic#72605) [Data] Query Input String manager (elastic#72093) [Security Solutions] Add tooltips (elastic#73436) Do not render descriptionless actions within an EuiCard (elastic#73611) ...

elasticmachine · 2021-09-23T14:28:39Z

Pinging @elastic/security-solution (Team: SecuritySolution)

cnasikas added Team:SIEM v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.9.0 labels Jul 28, 2020

cnasikas requested a review from patrykkopycinski July 28, 2020 12:21

cnasikas requested review from a team as code owners July 28, 2020 12:21

cnasikas self-assigned this Jul 28, 2020

cnasikas requested review from stephmilovic and MikePaquette July 28, 2020 12:22

Add tooltips

7d644d0

cnasikas force-pushed the add_tooltips branch from e985f2d to 7d644d0 Compare July 28, 2020 13:05

patrykkopycinski approved these changes Jul 28, 2020

View reviewed changes

andrew-goldstein reviewed Jul 28, 2020

View reviewed changes

...ck/plugins/security_solution/public/timelines/components/timeline/properties/translations.ts Outdated Show resolved Hide resolved

andrew-goldstein reviewed Jul 28, 2020

View reviewed changes

Hide tooltip when actions are active

62983f1

cnasikas requested a review from andrew-goldstein July 28, 2020 18:11

andrew-goldstein approved these changes Jul 28, 2020

View reviewed changes

Merge branch 'master' into add_tooltips

b03576e

Merge branch 'master' into add_tooltips

3b8d61a

Merge branch 'master' into add_tooltips

80e9514

Merge branch 'master' into add_tooltips

558b69c

Merge branch 'master' into add_tooltips

6fd6be3

andrew-goldstein merged commit b729446 into elastic:master Jul 29, 2020

andrew-goldstein mentioned this pull request Jul 29, 2020

[7.x] [Security Solutions] Add tooltips (#73436) #73627

Merged

andrew-goldstein mentioned this pull request Jul 29, 2020

[7.9] [Security Solutions] Add tooltips (#73436) #73628

Merged

cnasikas added a commit that referenced this pull request Jul 29, 2020

[7.x] [Security Solutions] Add tooltips (#73436) (#73627)

9ed8227

Co-authored-by: Christos Nasikas <christos.nasikas@elastic.co> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Sep 23, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Security Solutions] Add tooltips #73436

[Security Solutions] Add tooltips #73436

cnasikas commented Jul 28, 2020 •

edited

Loading

elasticmachine commented Jul 28, 2020

andrew-goldstein left a comment

cnasikas commented Jul 28, 2020 •

edited

Loading

andrew-goldstein left a comment

andrew-goldstein commented Jul 28, 2020

cnasikas commented Jul 28, 2020

andrew-goldstein commented Jul 28, 2020

andrew-goldstein commented Jul 29, 2020

andrew-goldstein commented Jul 29, 2020

kibanamachine commented Jul 29, 2020

elasticmachine commented Sep 23, 2021

[Security Solutions] Add tooltips #73436

[Security Solutions] Add tooltips #73436

Conversation

cnasikas commented Jul 28, 2020 • edited Loading

Summary

Checklist

For maintainers

elasticmachine commented Jul 28, 2020

andrew-goldstein left a comment

Choose a reason for hiding this comment

cnasikas commented Jul 28, 2020 • edited Loading

andrew-goldstein left a comment

Choose a reason for hiding this comment

andrew-goldstein commented Jul 28, 2020

cnasikas commented Jul 28, 2020

andrew-goldstein commented Jul 28, 2020

andrew-goldstein commented Jul 29, 2020

andrew-goldstein commented Jul 29, 2020

kibanamachine commented Jul 29, 2020

💚 Build Succeeded

Build metrics

async chunks size

History

elasticmachine commented Sep 23, 2021

cnasikas commented Jul 28, 2020 •

edited

Loading

cnasikas commented Jul 28, 2020 •

edited

Loading